CLI: Update hypeman SDK to v0.27.0 (3ea9a13) and add new commands/flags - #64
Open
kernel-internal[bot] wants to merge 8 commits into
Open
CLI: Update hypeman SDK to v0.27.0 (3ea9a13) and add new commands/flags#64kernel-internal[bot] wants to merge 8 commits into
kernel-internal[bot] wants to merge 8 commits into
Conversation
The SDK now accepts Docker-style registry credentials on ImageNewParams, so a private-registry image can be pulled without the server holding credentials for that registry. Expose them as --username/--password/--registry-token on `hypeman image create` and `hypeman pull`. `hypeman push create` already had the same three flags for the outbound push credentials, so both call sites now share one flag set and one builder. A full enumeration of the 61 SDK methods in api.md and their param struct fields against the CLI command tree found no other coverage gaps. Co-authored-by: Cursor <cursoragent@cursor.com>
Bump github.com/kernel/hypeman-go to 7f21c67d750f6dd66c6b6af04e88c710841f2daf, which adds the GET /capabilities resource. Expose it as `hypeman capabilities` so users can discover which runtimes and features a host actually supports instead of hard-coding hypervisor knowledge. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com> # Conflicts: # pkg/cmd/pushcmd.go
|
|
||
| if format == "auto" || format == "" { | ||
| return showCapabilities(os.Stdout, res) | ||
| } |
There was a problem hiding this comment.
Transform ignored in default format
Medium Severity
handleCapabilities always prints the full showCapabilities table when --format is the default auto, and never applies --transform. The advertised hypeman capabilities --transform runtimes path therefore still dumps the whole summary instead of falling through to ShowJSON.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit b91ebcf. Configure here.
Bumps github.com/kernel/hypeman-go to ea5a7f8d0d069b51e10189a8d0196f6155aaa4a8. A full enumeration of api.md against the pkg/cmd command tree found every SDK method already reachable from the CLI, but the SDK bump adds instance TTL / absolute expiration to InstanceNewParams and InstanceUpdateParams, which had no CLI surface. Expose them as --ttl and --expires-at on `hypeman run` and a new `hypeman update expiration` subcommand, rejecting the mutually exclusive combination and malformed values before the API round trip. Co-authored-by: Cursor <cursoragent@cursor.com>
Pin the CLI to the tagged v0.26.0 release of github.com/kernel/hypeman-go instead of the pseudo-version for the unreleased ea5a7f8d commit. A full enumeration of api.md methods and their param structs against the CLI command tree found no coverage gaps: all 62 SDK methods and every param field already map to an existing command or flag. Co-authored-by: Cursor <cursoragent@cursor.com>
Resolves the go.sum conflict by regenerating it against the current module graph, and bumps github.com/kernel/hypeman-go to v0.26.1 (392f55111d188fad255676e90c0c55d7c4b35b73). A full enumeration of api.md methods and their param structs against pkg/cmd found no coverage gaps, so no new commands or flags are needed. Co-authored-by: Cursor <cursoragent@cursor.com>
Bumps github.com/kernel/hypeman-go to 013dac97bc8391d82f4b583f1a579c3024f23f88,
which adds InstanceDeleteParams so callers can skip the graceful guest shutdown
that DELETE /instances/{id} performs by default.
Exposes the new field as `hypeman rm --graceful-shutdown`. The value is only
sent when explicitly set so the server default (true) still applies otherwise.
Compose teardown passes empty params to preserve its current behavior.
Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d650026. Configure here.
Full enumeration of api.md methods and CLI commands found no coverage gaps; the SDK change since 013dac97 is a release-only commit. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates the Hypeman Go SDK to
3ea9a139ea5b428182009fcbb39a0ca77d5bcd43(v0.27.0) and carries the CLI commands/flags added for new SDK methods on this evergreen branch.SDK Update
3ea9a139ea5b428182009fcbb39a0ca77d5bcd43(v0.27.0)Coverage Analysis
This PR was generated by performing a full enumeration of SDK methods and CLI commands.
api.mdwere enumerated and each has a corresponding CLI command (or a documented internal call site, e.g.client.Instances.GetbehindResolveInstance,client.Instances.Statbehindcp,client.Builds.Events/client.Instances.Logsvia their*Streamingvariants behindbuildandlogs).HealthCheckParam,AutoStandbyPolicyParam,SnapshotCompressionConfigParam, andIngressRuleParam) were checked against the CLI flag set. The only fields with no dedicated flag areInstanceNewParamsCredential.InjectandInstanceNewParamsCredentialInject.Hosts, which are reachable throughhypeman run --credentials-jsonsince that flag unmarshals the full credential policy map.openapi.yamlcontains no endpoints markedx-cli-skip: true.No new coverage gaps were introduced by this SDK version; the change since the previously vendored
013dac97is a release-only commit.New Commands
hypeman capabilitiesforclient.Capabilities.Get()hypeman update expirationforclient.Instances.Update()withInstanceUpdateParams.Ttl/ExpiresAtNew Flags
--graceful-shutdownonhypeman rmforInstanceDeleteParams.GracefulShutdown--ttland--expires-atonhypeman runforInstanceNewParams.TtlandInstanceNewParams.ExpiresAt--ttland--expires-atonhypeman update expirationforInstanceUpdateParams.TtlandInstanceUpdateParams.ExpiresAt--username,--password, and--registry-tokenonhypeman image createandhypeman pullforImageNewParams.Credentials(PushCredentialsParam.Username/Password/RegistryToken)Verification
go build ./...,go vet ./..., andgo test ./...all pass.Triggered by: kernel/hypeman-go@3ea9a13
Reviewer: @stainless-app[bot]